Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated api endpoint from /users/self to /users?profile=true #959

Conversation

Hariom01010
Copy link
Contributor

Date: 07 Dec 2024

Developer Name: Hariom Vashista


Issue Ticket Number:-

Real-Dev-Squad/todo-action-items#255 repo#2

Description:

Add description of the PR here
This PR aims to remove the deprecated API endpoint /users/self and update it with /users?profile=true
Is Under Feature Flag

  • Yes
  • No

Database changes

  • Yes
  • No

Breaking changes (If your feature is breaking/missing something please mention pending tickets)

  • Yes
  • No

Is Development Tested?

  • [Y] Yes
  • No

Tested in staging?

  • [Y] Yes
  • No

Add relevant Screenshot below ( e.g test coverage etc. )

@prakashchoudhary07
Copy link
Contributor

@Hariom01010 Please check, Tests are failing

@prakashchoudhary07 prakashchoudhary07 self-requested a review December 8, 2024 04:39
Copy link
Contributor

@prakashchoudhary07 prakashchoudhary07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests are faling?

@Hariom01010
Copy link
Contributor Author

Tests are faling?
I will look into it sir. I have checked the console.logs and found the failing test. I will see what's causing it and fix it and update you asap.

@Hariom01010
Copy link
Contributor Author

Hello Prakash sir, I tried to reproduce the failing tests as follows:

  1. Cloned the repo again.
  2. Made the changes.
  3. Run npm lint:fix
  4. Run ember test

Now all tests are passing for some reason. So i am not able to reproduce it. So i have created a new branch and will be closing this PR and create a new PR from that branch and the test result is OK there. Will attach the screenshot too sir.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants